Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pre-commit.ci] pre-commit autoupdate #8100

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Conversation

pyproject.toml Outdated
@@ -38,22 +40,21 @@ classifiers = [
dynamic = [
"version",
]
[project.optional-dependencies]
docs = [
optional-dependencies.docs = [
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pyproject-fmt version 2 has a fairly big change here, reformatting these from "expanded" to "inline".

I slightly prefer the old version, but am not too bothered really.

There's a chance pyproject-fmt might add an option to retain the old version. We could revert these changes for now, see if it's added by the next monthly autoupdate.

What do others think?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree that the old version is neater.

If this is just how things are with the latest version, that's fine.
If you think we're just going to revert this change in a month though, I'd rather keep it as it is for now than confusing the blame for this file.

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.4.3 → v0.4.7](astral-sh/ruff-pre-commit@v0.4.3...v0.4.7)
- [github.com/pre-commit/mirrors-clang-format: v18.1.4 → v18.1.5](pre-commit/mirrors-clang-format@v18.1.4...v18.1.5)
- [github.com/python-jsonschema/check-jsonschema: 0.28.2 → 0.28.4](python-jsonschema/check-jsonschema@0.28.2...0.28.4)
- [github.com/abravalheri/validate-pyproject: v0.16 → v0.18](abravalheri/validate-pyproject@v0.16...v0.18)
@hugovk hugovk force-pushed the pre-commit-ci-update-config branch from e10b711 to 322814d Compare June 4, 2024 07:51
@radarhere radarhere merged commit 1794a94 into main Jun 4, 2024
105 checks passed
@radarhere radarhere deleted the pre-commit-ci-update-config branch June 4, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants